-
Notifications
You must be signed in to change notification settings - Fork 470
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(addon-doc): add aria-label for dark mode button #9108
Conversation
Pull request was closed ✔️All saved screenshots (for current PR) were deleted 🗑️ |
Visit the preview URL for this PR (updated for commit a3a7231): https://taiga-previews--pr9108-splincode-demo-1-5pnbby5i.web.app (expires Fri, 20 Sep 2024 19:45:48 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 20bbff1ad19772ecf9c5134a84c0641b0d776f4a |
BundleMonFiles updated (1)
Unchanged files (4)
Total files change +26B 0% Groups updated (1)
Final result: ✅ View report in BundleMon website ➡️ |
@@ -11,6 +11,7 @@ | |||
<ng-content select="tuiDocHeader" /> | |||
<button | |||
appearance="secondary" | |||
aria-label="Switch between dark and light mode" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can just write this directly inside the button, that's exactly what we have font-size: 0 in icon buttons
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you mean about directly?
Before
After